chore: unify console app base path to / by Copilot · Pull Request #978 · objectstack-ai/objectui

Copilot · 2026-03-03T01:45:11Z

Console app Vite base defaulted to /console/ with a VITE_BASE_PATH env override, causing inconsistency between local dev and production. Unified to / everywhere.

Changes

apps/console/vite.config.ts — Hardcode base: '/', remove env var override
e2e/helpers/index.ts — CONSOLE_BASE → '' (all E2E URLs now root-relative)
apps/console/vercel.json — Drop redundant VITE_BASE_PATH=/ from build command
playwright.config.ts — webServer URL /console/ → /
apps/console/plugin.ts — JSDoc update
ROADMAP.md — P1.16 entry

// before
const basePath = process.env.VITE_BASE_PATH || '/console/';

// after
const basePath = '/';

Original prompt

This section details on the original issue you should resolve

<issue_title>Console app base 路径统一为 /</issue_title>
<issue_description>## 需求

当前 apps/console 的 Vite base 路径在本地调试和生产部署上未统一，默认用 /console/ 前缀。期望统一所有环境，无论本地开发还是生产打包/部署，base 均为 /，不带 /console 前缀。

具体方案：

修改 apps/console/vite.config.ts：

base 默认始终设为 /，无论 dev 还是 build。

所有部署、E2E、以及 vercel 配置均统一使用 /。

移除所有对 /console/ base 路径的依赖与多余注释。

包括 e2e/helpers/index.ts 的 CONSOLE_BASE，如需则默认 /。

检查 vercel/build 脚本和文档。

建议代码：
// vite.config.ts
export default defineConfig({
  base: '/',
  // ...其他配置
});
相关文件

apps/console/vite.config.ts

e2e/helpers/index.ts（如有）

apps/console/vercel.json

文档及注释

完成后请：

运行测试，确保开发、预览、生产部署均能在 / 主路径下正常访问

更新 ROADMAP.md

关联相关 Issue

如需跨 repo 协作请另建 cross-repo issue。</issue_description>

Comments on the Issue (you are @copilot in this section)

Fixes Console app base 路径统一为 / #977

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

vercel · 2026-03-03T01:45:16Z

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
objectui	Ready	Preview, Comment	Mar 3, 2026 1:53am
objectui-demo	Ready	Preview, Comment	Mar 3, 2026 1:53am
objectui-storybook	Ready	Preview, Comment	Mar 3, 2026 1:53am

- vite.config.ts: base path changed from /console/ to / - e2e/helpers: CONSOLE_BASE changed from /console to empty string - vercel.json: removed redundant VITE_BASE_PATH=/ override - playwright.config.ts: webServer url updated to / - plugin.ts: updated JSDoc comments - ROADMAP.md: added P1.16 entry Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>

Copilot

Pull request overview

Unifies the Console app’s Vite base path to / so local dev, preview/E2E, and Vercel deployments all resolve routes and assets from the root consistently.

Changes:

Hardcode Vite base: '/' and remove the VITE_BASE_PATH override path.
Update Playwright and E2E helper base URL/path assumptions to be root-relative.
Simplify Vercel build command by removing the redundant VITE_BASE_PATH=/ env var.

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
playwright.config.ts	Updates Playwright webServer URL from `/console/` to `/`.
e2e/helpers/index.ts	Switches `CONSOLE_BASE` to `''` to generate root-relative E2E URLs.
apps/console/vite.config.ts	Forces Vite `base` to `/` (no env override).
apps/console/vercel.json	Removes `VITE_BASE_PATH=/` from the build command.
apps/console/plugin.ts	Updates JSDoc to reflect the new base path.
ROADMAP.md	Adds P1.16 entry documenting completion of the base-path unification.

Copilot · 2026-03-03T02:32:06Z

apps/console/plugin.ts

 * HonoServerPlugin auto-discovers this plugin via `type: 'ui-plugin'`
 * and mounts it at `/<slug>` (i.e. `/console`).
 *
- * The SPA must be built with the matching base path:
- *   VITE_BASE_PATH=/console/ pnpm build
+ * The SPA is built with base path '/'.
 *


The JSDoc still states HonoServerPlugin mounts this UI at /<slug> (i.e. /console), but the comment now says the SPA is built with base path '/'. If the plugin is actually served under /console, a '/' base will generate root-absolute asset URLs (e.g. /assets/...) that won’t resolve under /console. Please update the JSDoc to accurately describe the serving path (or document how default=true affects mounting), so the base path and mount path stay consistent.

Copilot · 2026-03-03T02:32:06Z

e2e/helpers/index.ts

 * Base path for the console app.
- * Matches `base` in `apps/console/vite.config.ts` (defaults to '/console/').
+ * Matches `base` in `apps/console/vite.config.ts` (always '/').
 */
-export const CONSOLE_BASE = '/console';
+export const CONSOLE_BASE = '';


The comment says this matches Vite base (always '/'), but CONSOLE_BASE is set to an empty string. If the intent is “root-relative” URLs, consider updating the comment to clarify why this is '' (or set it to '/' and ensure call sites don’t produce double slashes).

Initial plan

2893f8e

Copilot AI assigned Copilot and hotlong Mar 3, 2026

Copilot started work on behalf of hotlong March 3, 2026 01:45 View session

vercel bot deployed to Preview – objectui-demo March 3, 2026 01:46 View deployment

Copilot AI and others added 2 commits March 3, 2026 01:47

fix: remove 'by default' from plugin.ts JSDoc per review feedback

d65bbcd

Co-authored-by: hotlong <50353452+hotlong@users.noreply.github.com>

vercel bot deployed to Preview – objectui-storybook March 3, 2026 01:49 View deployment

Copilot AI changed the title ~~[WIP] Update console app base path to unify across environments~~ chore: unify console app base path to / Mar 3, 2026

Copilot finished work on behalf of hotlong March 3, 2026 01:50

vercel bot deployed to Preview – objectui-demo March 3, 2026 01:51 View deployment

vercel bot deployed to Preview – objectui March 3, 2026 01:53 View deployment

hotlong marked this pull request as ready for review March 3, 2026 02:26

Copilot AI review requested due to automatic review settings March 3, 2026 02:26

hotlong merged commit d471d0f into main Mar 3, 2026
4 checks passed

Copilot started reviewing on behalf of hotlong March 3, 2026 02:27 View session

Copilot AI reviewed Mar 3, 2026

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

chore: unify console app base path to /#978

chore: unify console app base path to /#978
hotlong merged 3 commits intomainfrom
copilot/update-console-app-base-path

Copilot AI commented Mar 3, 2026 •

edited

Loading

Uh oh!

vercel bot commented Mar 3, 2026 •

edited

Loading

Uh oh!

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Mar 3, 2026

Uh oh!

Copilot AI Mar 3, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Conversation

Copilot AI commented Mar 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Changes

具体方案：

建议代码：

相关文件

Comments on the Issue (you are @copilot in this section)

Uh oh!

vercel bot commented Mar 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Mar 3, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Copilot AI commented Mar 3, 2026 •

edited

Loading

vercel bot commented Mar 3, 2026 •

edited

Loading